Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

SNOW-902588: Tests parallelization #772

Merged
merged 28 commits into from
Sep 12, 2023
Merged

SNOW-902588: Tests parallelization #772

merged 28 commits into from
Sep 12, 2023

Conversation

sfc-gh-pbulawa
Copy link
Collaborator

@sfc-gh-pbulawa sfc-gh-pbulawa commented Sep 4, 2023 •
edited
Loading

Description

Enable the parallelization of the test cases.

Checklist

  • Code compiles correctly
  • Code is formatted according to Coding Conventions
  • Created tests which fail without the change (if possible)
  • All tests passing (dotnet test)
  • Extended the README / documentation, if necessary
  • Provide JIRA issue id (if possible) or GitHub issue id in PR name

@sfc-gh-pbulawa
Enable parallelization
898295b
@sfc-gh-pbulawa
Set test to be non-parallelizable
53a6df1
@sfc-gh-pbulawa
Add non-parallelizable for the tests using global config
b692548
@sfc-gh-pbulawa
Revert the schema at the end of the test
b9ecf30
@sfc-gh-pbulawa
Set LevelOfParallelism parameter
c6e8c89
@sfc-gh-pbulawa
Change LevelOfParallelism
107d3c8
@sfc-gh-pbulawa
Clear pools after tests
cbc7bb6
@sfc-gh-pbulawa
Fix multithreading test
7703264
@sfc-gh-pbulawa
Clear pools after connection pool tests
3f5c32e
@sfc-gh-pbulawa
Set static for OneTimeTearDown
ef31891
@sfc-gh-pbulawa
Remove unnecessary cancellation tokens
9766ef5
@sfc-gh-pbulawa
Add cancellation token as it is required by .NET Framework
6736423
@sfc-gh-pbulawa
Set cancellation token for CloseAsync
88ec232
@sfc-gh-pbulawa
Remove NonParallelizable
95d85ec
@sfc-gh-pbulawa
Add NonParallelizable attribute for each test using MockSnowflakeDbCo…
cbd42ef 
…nnection
@sfc-gh-pbulawa
Merge remote-tracking branch 'origin/master' into SNOW-902588-test-pa…
3ba13c5 
…rallelization

# Conflicts:
#	Snowflake.Data.Tests/IntegrationTests/SFConnectionIT.cs
@sfc-gh-pbulawa
Remove unnecessary clear pools
98f12c5
@sfc-gh-pbulawa
Add one more test as non-parallelizable
0c9102e
@sfc-gh-pbulawa
Test output test
8f0c683
@sfc-gh-pbulawa
Fix test
b672918
@sfc-gh-pbulawa sfc-gh-pbulawa marked this pull request as ready for review September 6, 2023 11:57
@sfc-gh-pbulawa sfc-gh-pbulawa requested a review from a team as a code owner September 6, 2023 11:57
@sfc-gh-pbulawa sfc-gh-pbulawa merged commit 4609451 into master Sep 12, 2023
@sfc-gh-pbulawa sfc-gh-pbulawa deleted the SNOW-902588-test-parallelization branch September 12, 2023 15:22
@github-actions github-actions bot locked and limited conversation to collaborators Sep 12, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@sfc-gh-knozderko sfc-gh-knozderko sfc-gh-knozderko approved these changes

@sfc-gh-dprzybysz sfc-gh-dprzybysz sfc-gh-dprzybysz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@sfc-gh-pbulawa @sfc-gh-dprzybysz @sfc-gh-knozderko